Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[stable23] Check calendar URI length before creation #33784

Merged
merged 2 commits into from
Sep 27, 2022
Merged

[stable23] Check calendar URI length before creation #33784

merged 2 commits into from
Sep 27, 2022

Conversation

backportbot-nextcloud[bot]
Copy link

@backportbot-nextcloud backportbot-nextcloud bot commented Sep 1, 2022 •
edited by miaulalala
Loading

  • ⚠️ This backport had conflicts and is incomplete

backport of #33139

@backportbot-nextcloud backportbot-nextcloud bot added bug feature: caldav Related to CalDAV internals labels Sep 1, 2022
@backportbot-nextcloud backportbot-nextcloud bot added this to the Nextcloud 23.0.10 milestone Sep 1, 2022
github-advanced-security[bot]
github-advanced-security bot found potential problems Sep 1, 2022
View reviewed changes
apps/dav/lib/CalDAV/CalDavBackend.php Fixed Show resolved Hide resolved
apps/dav/lib/CalDAV/CalDavBackend.php Fixed Show resolved Hide resolved
@miaulalala miaulalala force-pushed the backport/33139/stable23 branch 2 times, most recently from dd22852 to e320bf8 Compare September 7, 2022 13:43
@miaulalala
Copy link
Contributor

@nickvergessen can I haz merge pls? :)

@nickvergessen
Copy link
Member

A rebase should allow merging?

miaulalala and others added 2 commits September 15, 2022 13:59
@miaulalala
Check calendar URI length before creation
ad0fca4 
Signed-off-by: Anna Larch <anna@nextcloud.com>
@nickvergessen @miaulalala
Check address book URI length before creation
992e05e 
Signed-off-by: Joas Schilling <coding@schilljs.com>
@miaulalala miaulalala force-pushed the backport/33139/stable23 branch from e320bf8 to 992e05e Compare September 15, 2022 11:59
@blizzz blizzz mentioned this pull request Sep 27, 2022
Merged
@blizzz blizzz merged commit 067f62c into stable23 Sep 27, 2022
@blizzz blizzz deleted the backport/33139/stable23 branch September 27, 2022 06:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@miaulalala miaulalala miaulalala approved these changes

@ChristophWurst ChristophWurst ChristophWurst approved these changes

@nickvergessen nickvergessen Awaiting requested review from nickvergessen

@tcitworld tcitworld Awaiting requested review from tcitworld

Assignees
No one assigned
Labels
bug feature: caldav Related to CalDAV internals
Projects
None yet
Milestone
Nextcloud 23.0.10
Development

Successfully merging this pull request may close these issues.
4 participants
@miaulalala @nickvergessen @ChristophWurst @blizzz